Skip to content

Conversation

mdboom
Copy link
Contributor

@mdboom mdboom commented Oct 25, 2024

This also fixes various things about the benchmarks so the unit tests pass on Windows.

@mdboom mdboom requested a review from kumaraditya303 October 25, 2024 15:44
@kumaraditya303
Copy link
Contributor

There's a failing CI of pypy which seems related to this

@mdboom
Copy link
Contributor Author

mdboom commented Oct 25, 2024

There's a failing CI of pypy which seems related to this

Yeah -- I don't think there is a version of dask that is new enough to be compatible with Windows but old enough to work with PyPy. I'm trying to figure out how to mark the benchmark as CPython-only, but I'm not sure there is a way.

Copy link
Contributor

@kumaraditya303 kumaraditya303 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mdboom mdboom merged commit 0741ae0 into python:main Oct 25, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants